Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[test] Add ut case to check hive clients pool deadlock #4493

Merged
merged 2 commits into from
Nov 11, 2024

Conversation

mxdzs0612
Copy link
Contributor

Purpose

Linked issue: close #4366

To test if hive client pool deadlock problem has been solved by #4468.

This test case will fail on release-0.9, however passes in master branch.

Tests

org.apache.paimon.hive.HiveCatalogTest#testListTablesLock

API and Format

no.

Documentation

no.

Copy link
Contributor

@JingsongLi JingsongLi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It seems it is hard to re-produce this deadlock...

@mxdzs0612
Copy link
Contributor Author

It seems it is hard to re-produce this deadlock...

Not that hard, just make 2 threads to call hiveCatalog.listTables at the same time, and assure that there is enough tables in the database, this stuck appears immediately. Just a little hard to construct this test case..

@JingsongLi
Copy link
Contributor

Thanks @mxdzs0612 , +1

@JingsongLi JingsongLi merged commit 800e26d into apache:master Nov 11, 2024
12 checks passed
@mxdzs0612 mxdzs0612 deleted the hive-lock-test branch November 11, 2024 08:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Bug] thread is blocked when executing show tables from catalog.database via DLF.
2 participants